Skip to content

Revert "Let generic-worker verify run-task/fetch-content integrity" #691

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
May 13, 2025

Conversation

jcristau
Copy link
Contributor

On top of the initial issue with the task schema not getting updated to match, it also causes failures because not all tasks have a direct dependency on the decision task, but generic-worker insists on that dependency existing when mounting an artifact from a task. So go back to mounting by url.

jcristau added 3 commits May 13, 2025 16:14
…askcluster#666)"

This causes failures because not all tasks have a direct dependency on the
decision task, but generic-worker insists on that dependency existing
when mounting an artifact from a task.  So go back to mounting by url.

This reverts commit f85ff32.
@jcristau jcristau requested a review from a team as a code owner May 13, 2025 14:17
@jcristau jcristau requested a review from hneiva May 13, 2025 14:17
Copy link
Contributor

@Eijebong Eijebong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Meant to click approve, don't have super strong feelings about keeping those

@jcristau jcristau changed the title Revert "Let generic-worker verify run-task/fetch-content integrity Revert "Let generic-worker verify run-task/fetch-content integrity" May 13, 2025
@jcristau jcristau merged commit 7f731f6 into taskcluster:main May 13, 2025
16 of 17 checks passed
@jcristau jcristau deleted the revert-gw-run-task-mount branch May 13, 2025 14:23
bhearsum added a commit to bhearsum/translations that referenced this pull request May 14, 2025
This picks up taskcluster/taskgraph#691, which is needed to revert back to the old GPU worker image.
bhearsum added a commit to bhearsum/translations that referenced this pull request May 15, 2025
This picks up taskcluster/taskgraph#691, which is needed to revert back to the old GPU worker image.
bhearsum added a commit to bhearsum/translations that referenced this pull request May 15, 2025
This picks up taskcluster/taskgraph#691, which is needed to revert back to the old GPU worker image.
eu9ene pushed a commit to mozilla/translations that referenced this pull request May 15, 2025
* backout switching gpu workers to docker + new worker image

* upgrade to taskgraph 14.2.1

This picks up taskcluster/taskgraph#691, which is needed to revert back to the old GPU worker image.
gregtatum pushed a commit to mozilla/translations that referenced this pull request May 16, 2025
This picks up taskcluster/taskgraph#691, which is needed to revert back to the old GPU worker image.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants